Skip to content

fix cache.NewSnapshot parameters#17545

Closed
huan wants to merge 1 commit intoenvoyproxy:mainfrom
huan:patch-1
Closed

fix cache.NewSnapshot parameters#17545
huan wants to merge 1 commit intoenvoyproxy:mainfrom
huan:patch-1

Conversation

@huan
Copy link

@huan huan commented Jul 30, 2021

# github.com/envoyproxy/go-control-plane/internal/example
internal/example/resource.go:165:26: not enough arguments in call to cache.NewSnapshot
	have (string, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource)
	want (string, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource)
make: *** [Makefile:79: bin/example] Error 2
ERROR: Service 'go-control-plane' failed to build: The command '/bin/sh -c cd go-control-plane && make bin/example' returned a non-zero code: 2

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link

Hi @huan, welcome and thank you for your contribution.

We will try to review your Pull Request as quickly as possible.

In the meantime, please take a look at the contribution guidelines if you have not done so already.

🐱

Caused by: #17545 was opened by huan.

see: more, trace.

```sh
# github.com/envoyproxy/go-control-plane/internal/example
internal/example/resource.go:165:26: not enough arguments in call to cache.NewSnapshot
	have (string, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource)
	want (string, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource, []types.Resource)
make: *** [Makefile:79: bin/example] Error 2
ERROR: Service 'go-control-plane' failed to build: The command '/bin/sh -c cd go-control-plane && make bin/example' returned a non-zero code: 2
```

Signed-off-by: Huan LI (李卓桓) <zixia@zixia.net>
@huan
Copy link
Author

huan commented Jul 31, 2021

Close this PR in favor of #17549 has been merged.

@huan huan closed this Jul 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant